Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Summary of ChangesHello @didwjdgns99, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 이 PR은 활동 등록 기능에서 날짜가 잘못 처리되던 버그를 수정하고, 이미지 업로드 컴포넌트의 사용자 인터페이스를 개선합니다. 특히, Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
| @@ -1,4 +1,4 @@ | |||
| import { Delete, PasswordHidden } from '@/assets/icons'; | |||
| import { Delete, FallbackImage, PasswordHidden } from '@/assets/icons'; | |||
There was a problem hiding this comment.
FallbackImage 아이콘으로 대체되면서 PasswordHidden 아이콘이 더 이상 사용되지 않습니다. import 문에서 제거하여 코드를 더 깔끔하게 유지하는 것이 좋습니다.
| import { Delete, FallbackImage, PasswordHidden } from '@/assets/icons'; | |
| import { Delete, FallbackImage } from '@/assets/icons'; |
| const schedules = values.rows.map((row) => { | ||
| const d = row.date; | ||
|
|
||
| const yyyy = d.getFullYear(); | ||
| const mm = String(d.getMonth() + 1).padStart(2, '0'); | ||
| const dd = String(d.getDate()).padStart(2, '0'); | ||
|
|
||
| return { | ||
| date: `${yyyy}-${mm}-${dd}`, // ✅ 로컬 기준 날짜 | ||
| startTime: row.startTime, | ||
| endTime: row.endTime, | ||
| }; | ||
| }); |
There was a problem hiding this comment.
날짜를 YYYY-MM-DD 형식으로 포맷하는 로직은 타임존 문제를 해결하는 올바른 수정입니다. 하지만 이 로직은 여러 곳에서 사용될 수 있으므로 유틸리티 함수로 추출하는 것을 고려해보세요. 리포지토리의 일반 규칙에 따르면, 순수 헬퍼 함수는 재사용성과 유지보수성을 높이기 위해 별도의 유틸리티 파일(예: utils/dateUtils.ts)에 두는 것이 좋습니다. 예를 들어, formatDateToYyyyMmDd(date: Date)와 같은 함수를 만들 수 있습니다. 또한, ActivityForm.tsx 파일에서도 비슷한 날짜 포맷 로직(toISOString().split('T')[0])이 사용되고 있는데, 이 역시 동일한 타임존 버그를 가질 수 있습니다. 공통 유틸리티 함수를 사용하면 이런 문제를 일관되게 해결하고 코드를 중앙에서 관리할 수 있습니다.
References
- Pure helper functions that do not depend on component state or props should be defined outside the component scope. For better organization and reusability, place them in a dedicated utility file (e.g.,
utils/timeUtils.ts).
1 participant
✅ PR 체크리스트
1. 코드 & 기능
🔗 이슈 번호
🎯 작업한 내용
💡 리뷰 요청 / 코멘트
🫧 참고사항
📸 스크린샷 (선택사항)